Skip to content

chore(ci): add actions:read and contents:write permissions to publish workflows (charmkeeper) - #124

Merged
florentianayuwono merged 2 commits into
mainfrom
charmkeeper/add-publish-workflow-permissions
Aug 19, 2026
Merged

chore(ci): add actions:read and contents:write permissions to publish workflows (charmkeeper)#124
florentianayuwono merged 2 commits into
mainfrom
charmkeeper/add-publish-workflow-permissions

Conversation

@weiiwang01

Copy link
Copy Markdown
Contributor

This PR adds actions: read and contents: write permissions to GitHub Actions jobs that use the canonical/operator-workflows/.github/workflows/publish_charm.yaml reusable workflow or canonical/charming-actions/upload-charm action.
These permissions are required for the workflow to function correctly with the latest changes to the reusable workflow.

@weiiwang01
weiiwang01 marked this pull request as ready for review May 28, 2026 16:38
@weiiwang01
weiiwang01 requested a review from a team as a code owner May 28, 2026 16:38
@weiiwang01
weiiwang01 requested review from DeeKay3 and yhaliaw and removed request for a team May 28, 2026 16:38
@weiiwang01 weiiwang01 changed the title chore(ci): add actions:read and contents:write permissions to publish workflows chore(ci): add actions:read and contents:write permissions to publish workflows (charmkeeper) May 28, 2026

@yhaliaw yhaliaw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved


🤝 Human review with AI assistance.

@florentianayuwono
florentianayuwono force-pushed the charmkeeper/add-publish-workflow-permissions branch from b0fba6b to a5c34ab Compare August 19, 2026 06:50
@florentianayuwono
florentianayuwono enabled auto-merge (squash) August 19, 2026 07:03
@florentianayuwono
florentianayuwono merged commit 6b7309f into main Aug 19, 2026
69 of 82 checks passed
@florentianayuwono
florentianayuwono deleted the charmkeeper/add-publish-workflow-permissions branch August 19, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants